Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move http:// to https:// for (mostly) external references #77

Merged
merged 3 commits into from
May 26, 2021
Merged

Move http:// to https:// for (mostly) external references #77

merged 3 commits into from
May 26, 2021

Conversation

paulo-ferraz-oliveira
Copy link
Contributor

@paulo-ferraz-oliveira paulo-ferraz-oliveira commented May 21, 2021

I saw I was getting redirected from https://rebar3.org (when clicking an external link) to a non-secure page.

I'm not sure using HTTP instead of HTTPS is done on purpose, but I guess it won't hurt, too much, to use HTTPS, where available (the "too much" part is important, since browsers will usually warn you if security is broken or compromised).

All individual links tested, which even resulted in some fixes to broken links (I'll point these out, in case the PR doesn't get accepted in this form, but link fixing is still up).

Notes:

@@ -20,7 +20,7 @@ In the above example the namespace `pc` (port compiler) has a provider named `co

We'll implement an example provider that "compiles" files in the directory `exc_files/` with the extension `.exc` to the priv directory of the application. The full source code can be found on [github](https://github.com/tsloughter/rebar3_ex_compiler).

The definitions are similar to what was done in the [Building Plugins](http://www.rebar3.org/docs/plugins) tutorial, but in this case we also have a `NAMESPACE` macro. This is important because the provider name is `compile` which, without defining a new namespace, would conflict with the existing `default` namespace `compile` provider.
The definitions are similar to what was done in the [Building Plugins](https://www.rebar3.org/docs/configuration/plugins/) tutorial, but in this case we also have a `NAMESPACE` macro. This is important because the provider name is `compile` which, without defining a new namespace, would conflict with the existing `default` namespace `compile` provider.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Link was broken.

@@ -25,7 +25,7 @@ $ rebar3 new cmake
===> Writing c_src/Makefile
```

In `test_nif`'s `rebar.config`, add the [pre_hooks](http://www.rebar3.org/v3.0/docs/configuration#hooks) line so that `make` is called when `compile` is run. Furthermore, add the `post_hooks` entry for cleaning up the built C object files.
In `test_nif`'s `rebar.config`, add the [pre_hooks](https://www.rebar3.org/docs/configuration/configuration/#hooks) line so that `make` is called when `compile` is run. Furthermore, add the `post_hooks` entry for cleaning up the built C object files.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Link was broken.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be a relative path

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@paulo-ferraz-oliveira
Copy link
Contributor Author

I wonder if references to rebar3.org could simple do without the prefix and go /configuration/..., for example.

Copy link
Collaborator

@ferd ferd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved, two internal links should be switched to actually be internal I think.


## Escript

Full details at the [escriptize command](http://www.rebar3.org/docs/commands#escriptize). Example configuration values below.
Full details at the [escriptize command](https://www.rebar3.org/docs/commands#escriptize). Example configuration values below.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should actually use relative paths here I think since it's a local URL.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yup. I'll move all of them, while I'm at it.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hm... actually some, while not broken, are pointing to a "parent" page, as per redirection rules, it seems. Will fix some more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All this is now done.

@@ -25,7 +25,7 @@ $ rebar3 new cmake
===> Writing c_src/Makefile
```

In `test_nif`'s `rebar.config`, add the [pre_hooks](http://www.rebar3.org/v3.0/docs/configuration#hooks) line so that `make` is called when `compile` is run. Furthermore, add the `post_hooks` entry for cleaning up the built C object files.
In `test_nif`'s `rebar.config`, add the [pre_hooks](https://www.rebar3.org/docs/configuration/configuration/#hooks) line so that `make` is called when `compile` is run. Furthermore, add the `post_hooks` entry for cleaning up the built C object files.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be a relative path

@@ -90,7 +90,7 @@ This configuration will allow to call `rebar3 check`, which will run the followi
- `xref` analysis for dead code and calls to functions that don't exist
- `dialyzer` checks for inconsistencies and type errors
- build the project documentation, to make sure it can do so without errors
- run regression tests in `proper` (using [the PropEr plugin](https://www.rebar3.org/docs/using-available-plugins#section-proper))
- run regression tests in `proper` (using [the PropEr plugin](/docs/configuration/plugins/#proper))
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wasn't broken, but was "no good".

@paulo-ferraz-oliveira
Copy link
Contributor Author

Hm... let me run a tool here, to see what else can be found.

@paulo-ferraz-oliveira
Copy link
Contributor Author

paulo-ferraz-oliveira commented May 21, 2021

Hm...

  • the HiPE thing I mentioned could be changed (⚠️ link IS actually broken)
  • the content still mentions the Basho mailing list (🚫 anything to do, here?)
  • https://hosting.review/web-hosting-glossary/#12 is broken (⚠️: I don't have an alternative for this)

@paulo-ferraz-oliveira
Copy link
Contributor Author

@ferd, given #77 (comment), do you think there's anything prevent this from being merge?

@ferd ferd merged commit 2fdb6ad into tsloughter:master May 26, 2021
@paulo-ferraz-oliveira
Copy link
Contributor Author

Ah... 😄

But there's still:

  • two broken links
  • mentions to Basho mailing list (is this correct?)

@ferd
Copy link
Collaborator

ferd commented May 26, 2021

The basho mailing list is still "active" and monitored. We'll eventually need to change freenode links to libera as well. I assumed the hosting.review thing was solved in a comment because I didn't see it in the changelog rather than assuming it was left otherwise. Let me take a look.

@ferd
Copy link
Collaborator

ferd commented May 26, 2021

#80

@paulo-ferraz-oliveira paulo-ferraz-oliveira deleted the http-to-https branch July 13, 2021 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants